Skip to content

【Hackathon 8th No.23】Improved Training of Wasserstein GANs 论文复现 #1152

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 15 commits into from
May 29, 2025
Merged

【Hackathon 8th No.23】Improved Training of Wasserstein GANs 论文复现 #1152

merged 15 commits into from
May 29, 2025

Conversation

XvLingWYY
Copy link
Contributor

PR types

PR changes

Describe

文档提交

@paddle-bot Paddle-bot
Copy link

paddle-bot bot commented May 12, 2025

Thanks for your contribution!

@luotao1 luotao1 mentioned this pull request May 13, 2025
Open
@XvLingWYY
Copy link
Contributor Author

老师这个文档有需要修改的地方吗？

@lijialin03
Copy link
Contributor

老师这个文档有需要修改的地方吗？

之前大致看了一下应该有一些格式上的问题，如果没看过这个 https://paddlescience-docs.readthedocs.io/zh-cn/latest/zh/development/#3 的话可以参考教程预览一下，具体的等代码合了我再看一下
代码那边有个检查卡住了，我联系一下相关同学

lijialin03
lijialin03 reviewed May 27, 2025
View reviewed changes
Copy link
Contributor

@lijialin03 lijialin03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

还有一点小问题，请再修改一下～

  1. 当前文档中所有应该显示代码的部分是空的，因为code代码合入了，所以麻烦重新拉一下代码，以检查代码显示有没有问题
  2. 模型评估应该有预训练模型（.pdparams），以及该预训练模型下的指标（参考其他文档），发给我我返回一个地址然后写到文档里，以及如果有图片也可以一并给我
  3. PaddleScience/README.txt中的“案例列表”部分也需要一并改一下

@XvLingWYY
Copy link
Contributor Author

  1. 当前文档中所有应该显示代码的部分是空的，因为code代码合入了，所以麻烦重新拉一下代码，以检查代码显示有没有问题

我下载最新的paddlescience修改后，这里代码都可以显示没有复现出问题

@XvLingWYY
Copy link
Contributor Author

2. 模型评估应该有预训练模型（.pdparams），以及该预训练模型下的指标（参考其他文档），发给我我返回一个地址然后写到文档里，以及如果有图片也可以一并给我

通过网盘分享的文件：models.zip
链接: https://pan.baidu.com/s/1ekP99D2ylox3yzqxQkJkyw?pwd=wgan 提取码: wgan
在里面是所有预训练模型需要链接，图片不需要

lijialin03
lijialin03 reviewed May 27, 2025
View reviewed changes
@XvLingWYY
Copy link
Contributor Author

谢谢老师，已完成所有修改

lijialin03
lijialin03 approved these changes May 27, 2025
View reviewed changes
Copy link
Contributor

@lijialin03 lijialin03 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@XvLingWYY XvLingWYY requested a review from lijialin03 May 28, 2025 10:31
@XvLingWYY
Copy link
Contributor Author

老师这个还需要修改吗？为什么还没有合入啊

HydrogenSulfate
HydrogenSulfate approved these changes May 29, 2025
View reviewed changes
Copy link
Collaborator

@HydrogenSulfate HydrogenSulfate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@HydrogenSulfate HydrogenSulfate merged commit 65a8798 into PaddlePaddle:develop May 29, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HydrogenSulfate HydrogenSulfate HydrogenSulfate approved these changes

@lijialin03 lijialin03 lijialin03 approved these changes

Assignees

@luotao1 luotao1

@lijialin03 lijialin03

Labels
contributor PaddlePaddle Hackathon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@XvLingWYY @lijialin03 @HydrogenSulfate @luotao1